Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI/CD doc #6424

Merged
merged 5 commits into from
Oct 25, 2024
Merged

Conversation

ryanlovett
Copy link
Collaborator

@ryanlovett ryanlovett commented Oct 25, 2024

Primarily, add the doc to the navigation. Then use markdown features where it makes sense. (terms, callout)

In-page headings should start with level 2 since the title is considered 1. Replace some syntactic sugar with semantic syntax. Also replace use of "here" links for accessibility.
@ryanlovett ryanlovett requested a review from shaneknapp October 25, 2024 20:14
@github-actions github-actions bot added documentation Issues around adding and modifying docs configuration labels Oct 25, 2024
@ryanlovett
Copy link
Collaborator Author

Looks like you added it to the nav while I was making this PR!

Copy link
Contributor

@shaneknapp shaneknapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, thanks! could you possibly add a little extra detail about quarto workflow and how that fits in to our CI/CD?

if you don't think it needs it, or it's described in detail elsewhere, that's fine.

@ryanlovett
Copy link
Collaborator Author

@shaneknapp Good idea, yes. I'll add something...

@ryanlovett ryanlovett merged commit 45a2459 into berkeley-dsep-infra:staging Oct 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues around adding and modifying docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants